Skip to content

Conversation

@metageeky
Copy link
Collaborator

Multiple fixes for the site to address what @phette23 noted in a different PR/branch.

- Add landmarks of main and header to all layouts
- Adjust skip links to work with new landmarks
- Improve color contrast on a few elements
git commit -am
@kaitlinnewson kaitlinnewson self-requested a review January 16, 2026 14:02
---
layout: secondary-nav
title: Conduct
title: Conduct & Safety
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
title: Conduct & Safety
title: 'Conduct & Safety'

{% include nav.html %}
</header>

<section>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just noting that removing this does create a minor styling change where the content shifts up (less spacing), in case this was not intentional.

Copy link
Member

@kaitlinnewson kaitlinnewson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@metageeky Just one minor comment and suggestion, otherwise it looks good!

@kaitlinnewson
Copy link
Member

One more thing I noticed while reviewing was that on the schedule pages, the times are inside of an h2 - not sure if that was intended, but could be addressed elsewhere.

Screenshot 2026-01-16 at 10 53 00 AM

@phette23 phette23 mentioned this pull request Jan 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants